-
Notifications
You must be signed in to change notification settings - Fork 15
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add missing types and parameters to Notifications API #332
Add missing types and parameters to Notifications API #332
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## master #332 +/- ##
============================================
- Coverage 45.98% 45.25% -0.74%
+ Complexity 481 477 -4
============================================
Files 129 126 -3
Lines 3523 3434 -89
Branches 231 232 +1
============================================
- Hits 1620 1554 -66
+ Misses 1727 1707 -20
+ Partials 176 173 -3
|
b8213ca
to
008f263
Compare
bigbone/src/main/kotlin/social/bigbone/api/entity/Notification.kt
Outdated
Show resolved
Hide resolved
Could we move forward with this one here? 😊 @andregasser @bocops |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, looks good to me! 👍
Again please 🥺 @andregasser |
Description
This PR adds missing parameters
types
andaccount_id
to the API to get all notifications.It also adds missing
NotificationType
s.To better handle future updates of the Mastodon API where we aren’t fast enough to release a new version, I have changed the fallback value to
null
instead ofMENTION
.I have changed the
Parameters
class a bit to improve testability. All unit tests still pass and outcome is as expected.For review, it could make sense to go commit by commit.
Closes #313
Type of Change
type
is now nullableHow Has This Been Tested?
Please describe the tests that you ran to verify your changes.
Mandatory Checklist
gradle check
and there were no errors reported